Reviewing files that changed from the base of the PR and between d0d97b7 and 5db1c0c.

• src/lib/accordion/custom.tsx (2 hunks)
• src/lib/accordion/index.tsx (2 hunks)
• src/lib/draggable-list/index.tsx (4 hunks)
• src/lib/form/bignumber-field/index.tsx (3 hunks)
• src/lib/form/bignumber-field/useBigNumberField.tsx (0 hunks)
• src/lib/index.ts (1 hunks)
• src/stories/draggable-list.stories.tsx (2 hunks)

• src/lib/form/bignumber-field/useBigNumberField.tsx

src/lib/index.ts (1)

18-18: LGTM! Form component export added successfully.

The export for the Form component is correctly placed among other form-related exports, following the established pattern in the file. This addition makes the Form component directly importable from the library's main entry point, improving accessibility and consistency.

src/lib/form/bignumber-field/index.tsx (3)

18-19: Good addition of proper documentation.

The JSDoc comment clearly explains the purpose of the new name prop, which is important for developers using this component.

---

34-34: LGTM! Name prop properly included in component props.

The name prop is correctly added to the component's parameter destructuring.

---

71-71: LGTM! Name prop correctly passed to Input component.

The name prop is properly passed down to the underlying Input component, enabling proper form integration and HTML form submission.

src/lib/accordion/index.tsx (2)

3-3: Nice import update for consistency.

The addition of isUndefined makes the code more explicit in how it handles undefined/null values.

---

31-33: Good enhancement in state initialization logic.

Replacing the nullish coalescing operator with an explicit isUndefined check makes the code more consistent with CustomAccordion and clarifies that both null and undefined values for defaultExpanded will result in using -1.

src/lib/accordion/custom.tsx (4)

3-3: Good utility import for consistent null/undefined handling.

Adding the isUndefined utility function import aligns with the approach used in the main Accordion component.

---

13-13: Nice enhancement with the defaultExpanded prop.

Adding this optional prop improves the component's flexibility by allowing control over which item is initially expanded.

---

19-19: Good prop handling in component signature.

The defaultExpanded prop is properly destructured from props.

---

22-24: Improved state initialization with explicit undefined check.

Using isUndefined for state initialization makes the code more explicit about handling both null and undefined values of defaultExpanded.

src/lib/draggable-list/index.tsx (5)

33-36: Well-documented new props with clear purposes.

The addition of dragDisabled and deletionDisabled flags with JSDoc comments clearly explains their purpose, making the component more flexible by allowing independent control over drag and delete operations.

---

52-53: Good default values maintaining backward compatibility.

Setting both flags to false by default ensures backward compatibility with existing usage of the component, as both drag and delete features remain enabled unless explicitly disabled.

---

85-85: Correct implementation of drag-and-drop disabling.

The implementation correctly prevents drag functionality by conditionally passing undefined instead of the drag-and-drop hooks when dragDisabled is true.

---

115-117: Proper visual indication of disabled drag functionality.

The drag handle icon is correctly hidden when drag functionality is disabled, providing a clear visual indication to users that the feature is unavailable.

---

121-121: Proper implementation of deletion disabling.

The delete button is correctly hidden when deletionDisabled is true, even when the item is hovered, preventing users from attempting to delete items when the feature is disabled.

src/stories/draggable-list.stories.tsx (1)

12-19: Good addition of Storybook controls for new props.

Adding argTypes for the new boolean props provides interactive controls in the Storybook UI, making it easy to test different component states.